Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

.storeAll now stores processed images for locals too, as it should be #810

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

HyperfocusDisordered
Copy link

@HyperfocusDisordered HyperfocusDisordered commented Aug 16, 2024

I needed this for correct behavior in my app...
Read closely:

/// Stores both processed images and the original image data.
    ///
    /// - note: If the RESOURCE IS LOCAL (has file:// or data:// scheme),
    /// only the PROCESSED IMAGES ARE STORED**.
    case storeAll

Same for .automatic
/// - note: Store only processed images for local resources (file:// or
/// data:// URL scheme).

Acutally it can be literally ∞ times more flexible if .shouldStoreResponseInDataCache becomes public settable lambda.
Lambda always enables Turing complete flexibility, no matter in which case...

PS i love this lib so much

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant